Upgrade/Install: Show and select translation updates on the Updates screen#11610
Upgrade/Install: Show and select translation updates on the Updates screen#11610s-a-s-k-i-a wants to merge 2 commits intoWordPress:trunkfrom
Conversation
|
Hi @s-a-s-k-i-a! 👋 Thank you for your contribution to WordPress! 💖 It looks like this is your first pull request to No one monitors this repository for new pull requests. Pull requests must be attached to a Trac ticket to be considered for inclusion in WordPress Core. To attach a pull request to a Trac ticket, please include the ticket's full URL in your pull request description. Pull requests are never merged on GitHub. The WordPress codebase continues to be managed through the SVN repository that this GitHub repository mirrors. Please feel free to open pull requests to work on any contribution you are making. More information about how GitHub pull requests can be used to contribute to WordPress can be found in the Core Handbook. Please include automated tests. Including tests in your pull request is one way to help your patch be considered faster. To learn about WordPress' test suites, visit the Automated Testing page in the handbook. If you have not had a chance, please review the Contribute with Code page in the WordPress Core Handbook. The Developer Hub also documents the various coding standards that are followed:
Thank you, |
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
|
Playground test passed. Manually tested and reviewed. Works. |
a43e354 to
7955a63
Compare
c8dd62d to
d9cbf3e
Compare
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
d9cbf3e to
c507717
Compare
Trac ticket: https://core.trac.wordpress.org/ticket/42281
Summary
list_translation_updates()currently renders only a generic translations notice and anUpdate Translationsbutton. Unlike the WordPress core, plugin, and theme update sections, it does not resolve or display the specific translation updates that are pending, and it does not allow individual translation updates to be deferred for later installation.This PR brings the translations section closer to the existing plugin and theme update UX by:
Changes
wp-admin/includes/update.phpwp-includes/update.phpupdate-core.phpwith a checkbox table styled like the existing plugin/theme update tablesLanguage_Pack_Upgrader::bulk_upgrade()WP_Automatic_Updater::should_update()andLanguage_Pack_Upgrader::async_upgrade()Test plan
php -l src/wp-admin/includes/update.phpphp -l src/wp-admin/update-core.phpphp -l src/wp-includes/update.phpphp -l src/wp-admin/includes/class-language-pack-upgrader.phpphp -l src/wp-admin/includes/class-wp-automatic-updater.phpphp -l tests/phpunit/tests/admin/includesUpdate.phpvendor/bin/phpcs --standard=phpcs.xml.dist src/wp-admin/includes/update.php src/wp-admin/update-core.php src/wp-includes/update.php src/wp-admin/includes/class-language-pack-upgrader.php src/wp-admin/includes/class-wp-automatic-updater.php tests/phpunit/tests/admin/includesUpdate.phpgit diff --checkhttps://wpcore.wp-studio.dev/wp-admin/update-core.phpusing a temporary MU-plugin fixture and local language-pack ZIP fixtures:3rows forWordPress,Hello Dolly, andTwenty Twenty-Four3translation rows were selected by default2successful translation updatesdeferred_translation_updatespersisted only the unchecked plugin translation updateWP_Automatic_Updater::should_update( 'translation', ... )returnedfalsefor the deferred plugin translation update andtruefor a non-deferred translation updateLanguage_Pack_Upgrader::async_upgrade()left the deferred translation update pendingTests_Admin_IncludesUpdatecould not be run in this environment because the local WordPress test configuration / Docker test environment was unavailableUse of AI Tools
AI assistance: Yes
Tool(s): Codex
Model(s): GPT-5-based Codex
Used for: implementation, test scaffolding, server-side E2E validation orchestration, and PR drafting; outputs were reviewed and validated with syntax checks, PHPCS,
git diff --check, and authenticated admin-page testing.This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.